requriments.txt?
and currently we are planning to integrate with the neon db and will look into it once we integrate it and look into the possibilities of merging the issue

@Harish-2003

ok , after integrating the db, anyhow all the db.py functions also require pydantic schemas in orm_mode for clean validation , i will add them and test the api routes and update the branch sir

if you could assign this schema validation issue to me , I would wait for the db integration too.

thank you for reviewing the pr

I made the existing code more enhanced and industry standard sir @Harish-2003

key improvements:

• created schemas folder with otp_schemas and user_schemas
• added status_codes to routes
• changed the query parameter to json body path parameters as phone number is PII and not advisable to use query parameters
• used the get_db func that is idle in database.py as dependency injection in routes and passed it as parameters to functions, which is more standard practice
• added empty init.py to folder for better modularity
• added access token verification functions and get current user functions in jwt.py for future use

I used postman to verify the routes and created a Twilio service to test all function, screenshots below with a one line description will helps you to understand the testing.

• checked /otp/register with random phone number

• checked send-otp function after creating a Twilio service free trail

• checked verify-otp function but with not registered phone number

• registered the crct phone number and verified the otp, otp got verified successfully ( changed the mail as previous mail raised unique constraint violation )

ss of otps received

FastAPI server side log

used a local .db file for quick testing and the visualization of db table is

from api side it all clean and clear to merge this pull request sir, as it is SQL Alchemy if we just change the database url to neon db , it works the same ! could you please look into this.

@Harish-2003